update packages for jammy, noble. (#1337)

create docker images with historical releases. (#1336)

* create docker images with historical releases.

* add scripts to automate image creation

* enhance test script to avoid cwd dependencies.

* move 1.7.0 base from focal to jammy

* make sure to clean up container

* make sure container is cleaned up.

* fix varaible use in arithmetic context

* delete unused archive from images.

* don't confuse our zlib and system zlib

* use system zlib shapelib for 1.8.0, 1.9.0

* add script to run GUI

* fix up 1.5.x plugins.

these were probematic for 1.5.4, previous releases ran ok with webkit.
translations don't work for 1.5.x

* shellcheck

* enhance GUI run script to forward LANG

* add bleeding edge image.

Merge pull request #1335 from GPSBabel/tidymore

clang tidy fixes

fix tidy modernize-use-nullptr

fix tidy readability-duplicate-include

fix tidy modernize-raw-string-literal

fix tidy readability-avoid-const-params-in-decls

optimize [QDate|QTime|QDateTime]::fromString format literals. (#1278)

* optimize [QDate|QTime|QDateTime]::fromString format literals.

These accept QStringViews in Qt6 which can be very efficiently
passed with UTF-16 character literals.

* optimize QDateTime::toString format literals.

This isn't new to Qt6, but we had a few we hadn't taken care of
yet.

* use QString::replace(QChar, QChar) instead of (QString, QString)

kill a position normilization clone caught by codacy. (#1333)

normalize waypoint position. (#1331)

prevent assertion failures in nmea reader (#1332)

* replace all empty nmea fields with 0.

including adjacent empty fileds, i.e. ",,," -> ",0,0,".

* defend against indexing past end of arrays.

prevents assertion failures in qstring.h operator[].

also consistently default longitude direction.

Humminbirdhash (#1327)

* refactor humminbird handling of wpt numbers.

also:
convert some macros to constexpr.
consistenly use strncpy to move strings to humminbird char arrays.`

* fix clang-diagnostic-missing-field-initializers

clang-tidy-18 fixes performance-unnecessary-copy-initialization (#1328)

Zap repeated examples (#1084)

On https://www.gpsbabel.org/htmldoc-development/fmt_globalsat.html
we observe these two examples are repeated further down, in context too. So they shouldn't also
appear higher up.

I am guessing that I have correctly zapped the higher up pair.

Co-authored-by: Robert Lipe

simplify osm_tag option processing. (#1325)

I suspect the check of "tagnd" was used in developement before
opt_tagnd was added.  The only thing the check did was ignore
opt_tag values that started with tagnd, they weren't used elsewhere.

add fix & test for humminbird route writer. (#1324)

The missing initialization was introduced in 824e01d.

update shapelib to 1.6.1 (#1323)

Modernize Humminbird internals (#1322)

* Modernize Humminbirds internals

---------

Co-authored-by: Robert Lipe <robertlipe@gpsbabel.org>

use implicit conversions for lat,lon degs<->rads in grtcirc interface (#1321)

* use implict conversions for lat,lon degs<->rads.

* use Position* return values.

* convert grtcirc i/f to PositionDeg, PositionRad.

* fix mac compile, denote converting ctor.

* inline PositionDeg/Rad converting ctors.

* respect PosotionX encapsulation.

* differentiate member names for PositionDeg, PositiionRad.  This can catch unintended usages where the type is not what was expected.

* clean up unneccesary usage of Waypoint::position.

* delete obsolete declaration.

* spell

* update garmin include

* consistenly unpack Positions to consts in grtcirc.

* provide default initializers for PositionX.

fix bend filter usage of linepart. (#1320)

* fix bend filter usage of linepart.

* fix new bend filter test.

* make new reference files valid kml.

* debug macos fails

* workaround rounding errors in problematic kml lookat range.

cleanup great circle. (#1318)

* cleanup great circle.

* fix whitespace

correct simplify filter with maximum allowable error option and the metric being overall route length or relative error (#1316)

* add test for simplify w/ relative+error opts.

* fix simplify filter error+length operation.

* fix simplify relative+error

* remove debug statement

* simplify error unit tests and documentation.

* format parse, update includes.

* prune redundant test case

have jeeps use std::endian (#1315)

* have jeeps use std::endian

* detect unsupported endian platforms.

move floor to C++20 (#1206)

* move floor to c++20

drop Visual Studio 2017.
use default member intializers for bit-fields.

* use more bit-field default initializers

* try Xcod 14.3.1 on macOS 13.

* echo clang version

which can be useful with homebrew fails_with

* use std::numbers

* finish eradication of M_PI.

* fix some whitespace

use constexpr for constants, match parms in declaration and definition. (#1314)

* use constexpr for constants, match parms in declaration and definition.

* increase precision of GPS_PI.

Use Helmert transform for conversions between OSGB36 and WSG84 (#1311)

* use Helmert transform for OSBG36 <-> WGS84.

* adjust unicsv_grids reference for Helmert xform.

* update copyright.

* use constexpr to select helmert inversion method.

* tweak helmert.

spell Datum (#1313)

unify access to ellipsoid parameters (#1312)

* define constants for often used spherioid parms.

* add function to calculate semi-minor axis.

* use semi major axis function more.

* have semi_minor_axis consume an ellipse

* compute semi minor axis in GPS_Ellipse class.

add constants for other ellipses that are used directly.

correct WGS84 semi minor axis value. (#1310)

It appears the GSR80 value was used instead of the WGS84 value, which
is a very small error.
However, there was a typo in the value in GPS_Math_XYZ_To_WGS84LatLonH
that resulted in an order of magnitude error!
This could lead to GPS_Math_XYZ_To_WGS84LatLonH never converging.
This may be why skytraq created their own function to convert ECEF
to WGS84.

cleanup datetime.h tidy and g++ warnings. (#1309)

Wsign-conversion
readability-implicit-bool-conversion
readability-else-after-return
cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers

enhance validate filter to report on track segments. (#1308)

use a signed type for waypt_counts and route counts. (#1307)

* use a signed type for waypt_counts and route counts.

Although larger in Qt6, the underlying type is signed.

* fix -Wformat-signedness warnings introduced by this PR.

* fix new -Wsign-conversion warnings.

teach lowranceusr about track segments (#1306)

* teach lowranceusr about track segments.

lowranceusr versions 2 & 3 have a continuous flag used to
indicate dicontinuities in a trail.
The lowranceusr version 2 & 3 reader now flags trail discontinuities
as new track segements.
The lowranceusr version 2 & 3 writer now flags new track segments as
discontinuities.

Also fix two subtle bugs:
1) when reading a version 2 or 3 trail with the break option we would
fail to break the trail if the discontinuity was at the start of a
section other than the first section.
2) when writing a version 2 or 3 trail with the merge option if the
number of points exceeded MAX_TRAIL_POINTS we would write a header
that indicated MAX_TRAIL_POINTS but actually write MAX_TRAIL_POINTS + 1.

* add lowranceusr test for track segments.

Update Russian translation (#1304)

* Update Russian translation

* Update Russian translation

correct garmin real time altitude, add geoid height. (#1303)

correct computation of timestamp with garmin real time positioning. (#1302)

* correct computation of timestamp with garmin real time positioning.

* tweak garmin pvt conversion

fix gpsapp copy_char_array that could leave part of a packet uninitialized. (#1301)

* fix gpsapp copy_char_array that could leave part of a packet uninitialized.

* tidy up a bit.

* delete empty statement

fix memory leaks writing garmin routes. (#1299)

enhance garmin route write for devices using 16 member icon set. (#1298)

enhance garmin route writes for devices using D201_Rte_Hdr_Type. (#1297)

which use GPS_PWay->rte_cmnt for the route name.

don't send links with GPS_A200_Send. (#1295)

fix faulty assertion in garmin. (#1290)

the '-' character is legal for some devices, but was
excluded in the assertions as it can be a metacharacter.

remove qtci installer code (#1289)

* remove qtci which hasn't been used for a long time.

* remove another obsolete qtci file.

* remove another qtci reference.

* delete obsolete travis build code.

* rename osx install script

* catch ci up with osx travis install script renames.

* remove obsolete debug code from qtio installer.

exclude qt .ts files from codacy analysis. (#1288)

they confuse eslint which thinks they are TypeScript.

Fix unicsv reader delimiter detection (#1287)

* Fix unicsv reader delimiter detection.

Any delimiter sequence that is enclosed has never been considered
a delimiter when splitting a line into fields.  However, previously
an enclosed potential delimiter could be detected when scanning the
first line, and then erroneously used as the delimiter.

* tweak auto

* add unicsv delimiter detection test

delete obsolete jeeps functions that were never used (#1285)

since introduction in 2002.

update macos release to Qt 6.5.3 (#1284)

refactor interface to jeeps pvt related commands. (#1283)

fix xcsv RECORD_DELIMITER CR. (#1282)

add Qt's webengine translations to package. (#1277)

tidy readability-convert-member-functions-to-static (#1276)

clang-tidy modernize-loop-convert (#1275)

clang-tidy readability-isolate-declaration (#1274)

Move floor to Qt 6.2 (#1272)

* move floor from Qt 5.15 to 6.2.

* fix some CI failures

* try harder to fix coverage build

* try to fix coverage again

* try agin

* move up msvc tools minimum for Qt6

* clean up clazy warnings with Qt6.

* drop redundnant CI job

* revert translation related unintentionl changes

* simplify find qt packages

* assume Qt6 in textstream

* move CI tidy to noble

* assume Qt6 in textstream.h

* remove obsolete makesetup.bat

* remove Qt5 support from coretool

* strip Qt5 from noble image

* update default qt versions.

* update minimum cmake version to match Qt 6.2.4 QT_SUPPORTED_MIN_CMAKE_VERSION_FOR_USING_QT

* assume QHash result is type size_t

* remove unnecessary qOverloads (in Qt6).

add Ubuntu noble to CI (#1273)

* add ubuntu noble docker support

* update qt6 packages for noble.

noble has some new qt6 packages that alleviate the need to
directly install some libraries.
also, the dependencies of some qt6 packages have been improved
allowing us to specify what we use more directly.
this results in the same set of packges being installed.

* remove dependent qt5 package

* add noble to CI

add fedora 40 to CI (#1270)

support builds with shapelib (as documented). (#1268)

update libusb to 1.0.27 (#1259)

work around homebrew python linking issues (#1267)

* debug python brew issues

* dbg2

* dbg3

* dbg4

* insulate workflow from homebrew location.

Add test coverage for ozi color setting options. (#1266)

* Unit test ozi color commandline options.

Improves test coverage.

Co-authored-by: Robert Lipe

update macos github workflow (#1265)

* update github macos runners.

macos-11 runner is now deprecated.

* update xcode versions for github

* update XML_CATALOG_FILES for macos-14 runner

* work around macos-14 runner brew locations

* find homebrew gsed on apple silicon

* generalize brew location detection in macos workflow

* update python version for arm availability

* push xcode versions to latest minor levels.

clang tidy 18 fixes readability-redundant-casting (#1262)

update zlib to 1.3.1 (#1258)

* update zlib to 1.3.1

* delete extra zlib files

update shapelib to 1.6.0 (#1257)

Bump actions/cache from 3 to 4 (#1256)

Bumps [actions/cache](https://github.com/actions/cache) from 3 to 4.
- [Release notes](https://github.com/actions/cache/releases)
- [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md)
- [Commits](https://github.com/actions/cache/compare/v3...v4)

---
updated-dependencies:
- dependency-name: actions/cache
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

update codacy badge (#1251)

update codeql workflow template (#1250)

Bump actions/upload-artifact from 3 to 4 (#1249)

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 3 to 4.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](https://github.com/actions/upload-artifact/compare/v3...v4)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump github/codeql-action from 2 to 3 (#1248)

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 2 to 3.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/v2...v3)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

modernize google maps javascript (#1247)

to avoid:
"js: Google Maps JavaScript API has been loaded directly without a callback. This is not supported and can lead to race conditions and suboptimal performance. For supported loading patterns please see https://goo.gle/js-api-loading"

remove CMakeSettings.json (#1246)

which had machine specific windows configuration values.

Bump actions/setup-python from 4 to 5 (#1245)

Bumps [actions/setup-python](https://github.com/actions/setup-python) from 4 to 5.
- [Release notes](https://github.com/actions/setup-python/releases)
- [Commits](https://github.com/actions/setup-python/compare/v4...v5)

---
updated-dependencies:
- dependency-name: actions/setup-python
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

make xmlgeneric a class (#1244)

* make xmlgeneric a class.

* encapsulate more of xmlgeneric

* use scoped enum

* encapsulate a bit more.

* retire xg_string type

* use scoped enums

* rule of zero applied to xmlgeneric class.

* remove test code

fix errors in gpi writer related to bitmaps. (#1243)

Erroneous warnings "garmin_gpi: Code error in load_bitmap_from_file,
expected output size 1060, actual output -28."

Corrupted colors when input bitmap has 24 bits per pixel.

bring static functions into googltakeout class. (#1240)

and other tidy modernizations.

straggling nits (#1236)

* actually quit using gpsfmt.h

* use auto

* empty statements

* join declaration and assignment

* fix tab

* make member funct static

* delete renamed prototypes

* const pointers

* more const pointers

* empty statements

misc cleanups (#1235)

simplify usage of homebrew on CI macos-11. (#1234)

homebrew installs are taking an extreme amount of time on macos-11,
with the following message:
  Warning: You are using macOS 11.
  We (and Apple) do not provide support for this old version.
  It is expected behaviour that some formulae will fail to build in this old version.
  It is expected behaviour that Homebrew will be buggy and slow.
To alleviate this pain skip the installation of jing and all it's
dependencies on macos-11.

retire LegacyFormat class. (#1233)

convert mtk formats to Format class. (#1226)

convert ozi to Format class. (#1232)

* convert ozi to Format class.

There is a subtle difference in the writer, the index is now
reset for every invocation.  This only matters if the writer is
used more than once. I believe this was the intent.

* fool with QIODevice and it's inheritence of QIODeviceBase.

convert garmin_txt to Format class (#1231)

* convert garmin_txt to Format class

* fix includes for qt5

convert garmin_xt to Format class (#1230)

convert tpg to dynamic Format class. (#1229)

convert gtm to Format class. (#1228)

convert v900 to dynamic Format class. (#1227)

convert tpo formats to Format class (#1225)

* convert tpo to Format class.

* capture comments

fix leak in jeeps serial comm on posix. (#1223)

see #1221

convert garmin to Format class (#1222)

* convert garmin to format class.

* add new file garmin.h

* make some member fns static.

* init member var

astyle jeeps included files (#1219)

gpsmath isolation (#1218)

gpsmath/gpsdatum/gpsproj don't have dependencies
on the rest of jeeps.

cleanup typedefs, macros, const tables.

use c++11 fixed width integer types in jeeps. (#1217)

* use c++11 fixed width integer types in jeeps.

thanks resharper!

* fix fixed width integers resharper missed.

in non windows code.

use uniform header guards in jeeps. (#1216)

and
delete an obsolete prototype.
rename jgpsutil.cc now that it doesn't conflict with a format.

jeeps function pointers, GPS_PPacket const correctness, GPS_Serial_[OSP]Packet -> GPS_Serial_Packet (#1215)

* use using for jeeps function prototypes.

* const correctness wrt GPS_PPacket.

The important bit here is in gpssend.cc, Build_Serial_Packet used to rececive a 1KB packet by copy, now it is by const reference.

* delete accidentally committed file.

* rename GPS_PPacket -> GPS_Packet.

original jeeps used GPS_SXXX, GPS_OSXXX, GPS_PXXX for the structure
tag, a typedef of the structure, and a pointer to a typedef of the
structure.  In the case of GPS_PPacket this convention was no longer
followed.

* change GPS_Packet back to a structure.

* try harder to change GPS_Packet back to a struct.

* clean up GPS_Serial_[SOP]Packet.

* catch some missed renames up.

clion on linux missed some windows only renames.

clang tidy fixes (#1214)

* fix tidy readability-named-parameter

* fix readability-named-parameter fix

* fix some readability-inconsistent-declaration-parameter-name tidy warnings.

* update tidy script to

ignore strptime
ignore third party headers
clean build directory
generate summary

* update git ignores

misc cleanups (#1213)

initialize garmin_fs class member.
catch gui up with Qt 5.15 floor.
rename tidy build directory.

make qstarz format dynamic (#1212)

add fedora 39 to ci (#1211)

* add fedora 39 to ci

* retire qAsConst for std::as_const

convert vcf to dynamic Format. (#1210)

refactor garmin_fs class (#1203)

* use container for garmin_fs ilinks.

and leave memory management up to the copy on write container.

* make gmsd const for read only usage.

this prevents the possiblity of detachment of the ilinks list.

* incorporate some variables and functions into garmin_fs_t

* unify garmin category conversions.

* fiddle with test time zone sensitivity.

* add utc option for garmin_txt reader.

* correct sign of adjustment

* update garmin_txt includes.